Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

perf: normalize LMS vs. Studio debug-toolbar settings #36134

Merged
merged 1 commit into from
Jan 21, 2025
Merged

perf: normalize LMS vs. Studio debug-toolbar settings #36134

merged 1 commit into from
Jan 21, 2025

Conversation

ormsbee
Copy link
Contributor

@ormsbee ormsbee commented Jan 18, 2025 

This change disables the profiling panel for performance reasons. It's
rarely useful anyway, given the lack of granularity in the data it
displays.

This commit also enables the Cache panel, which is import for tracking
where we're making excessive calls to redis/memcached.

@ormsbee ormsbee added the performance Relates to improving latency/throughput or reducing resource usage label Jan 18, 2025
@ormsbee ormsbee force-pushed the perf-enable-cache-panel branch from 48b7545 to 44f703f Compare January 18, 2025 21:16
@ormsbee ormsbee mentioned this pull request Jan 19, 2025
Open
9 tasks
@ormsbee
perf: normalize LMS vs. Studio debug-toolbar settings
6c78965 
This change disables the profiling panel for performance reasons. It's
rarely useful anyway, given the lack of granularity in the data it
displays.

This commit also enables the Cache panel, which is import for tracking
where we're making excessive calls to redis/memcached.
@ormsbee ormsbee force-pushed the perf-enable-cache-panel branch from 44f703f to 6c78965 Compare January 21, 2025 15:29
@ormsbee ormsbee merged commit 7635b8e into openedx:master Jan 21, 2025
49 checks passed
@ormsbee ormsbee deleted the perf-enable-cache-panel branch January 21, 2025 16:03
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

1 similar comment
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

1 similar comment
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@feanil feanil feanil approved these changes

Assignees
No one assigned
Labels
performance Relates to improving latency/throughput or reducing resource usage
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ormsbee @edx-pipeline-bot @feanil